Skip to content

[Doctrine] Replace deprecated Doctrine\ORM\Event\LifecycleEventArgs #18145

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 5, 2023
Merged

[Doctrine] Replace deprecated Doctrine\ORM\Event\LifecycleEventArgs #18145

merged 1 commit into from
Apr 5, 2023

Conversation

sdespont
Copy link
Contributor

@sdespont sdespont commented Mar 30, 2023

Replace deprecated Doctrine\ORM\Event\LifecycleEventArgs by dedicated events :

  • Doctrine\ORM\Event\PostPersistEventArgs;
  • Doctrine\ORM\Event\PostRemoveEventArgs;
  • Doctrine\ORM\Event\PostUpdateEventArgs;

https://github.com/doctrine/orm/blob/2.14.x/UPGRADE.md#deprecated-doctrineormeventlifecycleeventargs-class

… events

Replace deprecated Doctrine\ORM\Event\LifecycleEventArgs by dedicated events : 
- Doctrine\ORM\Event\PostPersistEventArgs;
- Doctrine\ORM\Event\PostRemoveEventArgs;
- Doctrine\ORM\Event\PostUpdateEventArgs;
@carsonbot carsonbot changed the title Replace deprecated Doctrine\ORM\Event\LifecycleEventArgs [Doctrine] Replace deprecated Doctrine\ORM\Event\LifecycleEventArgs Apr 5, 2023
@javiereguiluz javiereguiluz added this to the 6.2 milestone Apr 5, 2023
@javiereguiluz javiereguiluz merged commit 0bc9fda into symfony:6.2 Apr 5, 2023
@javiereguiluz
Copy link
Member

Sébastien, thanks and congrats on your first Symfony Docs contribution 🎉

Note that while merging we made some minor tweaks to add an explicit mention to the deprecated LifecycleEventArgs. See 655fbe0 We usually do that when something well-known is deprecated. This way, readers and people using the search engine, can still look for LifecycleEventArgs and find this article. Thanks!

@sdespont sdespont deleted the patch-1 branch April 5, 2023 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants